Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

#973 Rails Admin authorization #977

Closed
wants to merge 2 commits into from
Closed

#973 Rails Admin authorization #977

wants to merge 2 commits into from

Conversation

orenyk
Copy link
Contributor

@orenyk orenyk commented Sep 17, 2014

Resolves #973, depends on #975 to work properly (rename the ability to cannot :access, :rails_admin here)

@aaron-effron
Copy link
Contributor

Also want to make sure I understand exactly what is being done here - do we want the admin menu at the top to be just be viewable when we are on the page as an admin? If so, the effects of the code look good to me.

@orenyk
Copy link
Contributor Author

orenyk commented Sep 22, 2014

This issue relates to access to Rails Admin, our administrative tool where superusers can go in and modify records directly. It's accessible via the /admin route, and we want to make sure that only superusers can go there. Once you verify that this prevents anyone other than superusers from accessing /admin then we can pull this in.

@orenyk
Copy link
Contributor Author

orenyk commented Oct 30, 2014

marking reviewed based on conversation w/ @aaron-effron

@orenyk
Copy link
Contributor Author

orenyk commented Oct 30, 2014

This has been superseded by changes in #2, closing and #973 will be closed w/ #1032.

@orenyk orenyk closed this Oct 30, 2014
@orenyk orenyk deleted the 973_access_control branch November 25, 2014 05:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set up access control to Rails Admin
2 participants